Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

(PDK-1527) Handle pdk new module --skip-interview without module name #788

Merged
merged 1 commit into from
Oct 23, 2019
Merged

(PDK-1527) Handle pdk new module --skip-interview without module name #788

merged 1 commit into from
Oct 23, 2019

Conversation

rodjek
Copy link
Contributor

@rodjek rodjek commented Oct 23, 2019

Exit cleanly with an error when a user tries to run pdk new module --skip-interview without providing a module name in the CLI args.

@rodjek
(PDK-1527) Handle pdk new module --skip-interview without module name
4697030 
Exit cleanly with an error when a user tries to run `pdk new module
--skip-interview` without providing a module name in the CLI args.
@rodjek rodjek added this to the November 2019 milestone Oct 23, 2019
@rodjek rodjek requested a review from a team as a code owner October 23, 2019 23:03
@rodjek rodjek added the bug label Oct 23, 2019
scotje
scotje approved these changes Oct 23, 2019
View reviewed changes
Copy link
Contributor

@scotje scotje left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@coveralls
Copy link

Coverage Status

Coverage increased (+0.004%) to 91.841% when pulling 4697030 on rodjek:pdk-1527 into 73149c5 on puppetlabs:master.

@rodjek rodjek merged commit 3b41015 into puppetlabs:master Oct 23, 2019
@rodjek rodjek deleted the pdk-1527 branch October 23, 2019 23:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@scotje scotje scotje approved these changes

Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
November 2019
Development

Successfully merging this pull request may close these issues.
3 participants
@rodjek @coveralls @scotje